Skip to content

feat(asset): add unassociated asset page#308

Merged
TylerAdamMartinez merged 4 commits into
stagingfrom
feature/tam-bdms-922-disassociated-list
Jul 2, 2026
Merged

feat(asset): add unassociated asset page#308
TylerAdamMartinez merged 4 commits into
stagingfrom
feature/tam-bdms-922-disassociated-list

Conversation

@TylerAdamMartinez

@TylerAdamMartinez TylerAdamMartinez commented Jul 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • Provides a dedicated Unassociated Assets page where administrators can review, preview, delete, or reassociate orphaned assets.

How

Implementation summary - the following was changed/added/removed:

  • Added a new Unassociated Assets page at /ocotillo/asset/unassociated.
  • Updated existing asset preview components to consume the shared AssetActions component.
  • Added a full-screen asset preview experience for the unassociated assets list.

Notes

Any special considerations, workarounds, or follow-up work to note?

  • The page is intended as a cleanup workflow for orphaned assets, allowing administrators to either permanently delete assets or reassociate them with an existing well.
  • The list uses server-side pagination (10 assets per page) and periodically refreshes to ensure signed URLs remain valid while users are reviewing assets.

@TylerAdamMartinez TylerAdamMartinez self-assigned this Jul 1, 2026
@TylerAdamMartinez TylerAdamMartinez added the enhancement New feature or request label Jul 1, 2026
@TylerAdamMartinez TylerAdamMartinez force-pushed the feature/tam-bdms-922-disassociated-list branch from 234bd1b to 8d31d08 Compare July 1, 2026 18:15
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-tam-bdms-922-disassociated-list-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

1 similar comment
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-tam-bdms-922-disassociated-list-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-tam-bdms-922-disassociated-list-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez TylerAdamMartinez force-pushed the feature/tam-bdms-922-disassociated-list branch from a2670b3 to 903361b Compare July 1, 2026 19:20
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-tam-bdms-922-disassociated-list-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-feature-tam-bdms-922-disassociated-list-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez TylerAdamMartinez marked this pull request as ready for review July 1, 2026 20:01
chasetmartin
chasetmartin approved these changes Jul 1, 2026
View reviewed changes

@chasetmartin chasetmartin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is working as expected for me and looks good. I think it's worth testing this with users to see if an overarching assets list page is needed or if this is plenty. Nice work

@TylerAdamMartinez TylerAdamMartinez merged commit efb0bdf into staging Jul 2, 2026
4 of 5 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the feature/tam-bdms-922-disassociated-list branch July 2, 2026 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chasetmartin chasetmartin chasetmartin approved these changes
@jirhiker jirhiker Awaiting requested review from jirhiker
@lizalino lizalino Awaiting requested review from lizalino

Assignees

@TylerAdamMartinez TylerAdamMartinez

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TylerAdamMartinez @chasetmartin